Skip to content

Enhancement/nested telemetry protos#9489

Draft
oscgonfer wants to merge 125 commits intomeshtastic:developfrom
fablabbcn:enhancement/nested-telemetry-protos
Draft

Enhancement/nested telemetry protos#9489
oscgonfer wants to merge 125 commits intomeshtastic:developfrom
fablabbcn:enhancement/nested-telemetry-protos

Conversation

@oscgonfer
Copy link
Contributor

@oscgonfer oscgonfer commented Jan 30, 2026 •
edited
Loading

Rebased onto #7198 (and #7245) for demo purposes. Still draft as it may make breaking changes in other clients if accepted.

This is a proposal to make Telemetry with nested protos (now only in AQ telemetry, but can be expanded to other telemetry modules) to add information regarding sensors in the payload (i.e. who is producing the data).

=== Meshtastic Packet Info ===
🔑 Device ID: 3665040172
📟 Device Name: Oscar Test
🗂️ Packet ID: 509853187
⏰ Timestamp: 2026-01-30T14:31:14Z
📡 Port Number: 67
📦 Full protobuf payload:
{ 'time': 1769783474,
  'air_quality_metrics': { 'sen5xdata': { 'pm10_standard': 1,
                                          'pm25_standard': 2,
                                          'pm40_standard': 2,
                                          'pm100_standard': 2,
                                          'particles_05um': 1200,
                                          'particles_10um': 200,
                                          'particles_25um': 0,
                                          'particles_40um': 0,
                                          'particles_100um': 0,
                                          'pm_temperature': 17.755,
                                          'pm_humidity': 68.51,
                                          'pm_voc_idx': 103.0,
                                          'pm_nox_idx': 1.0,
                                          'particles_tps': 0.581},
                           'scd4xdata': { 'co2': 1354,
                                          'co2_temperature': 15.33608,
                                          'co2_humidity': 81.16884}}}

Related to #9474 and building on protobufs proposal in meshtastic/protobufs#852.

An additional improvement would be to add other values (serial number, or sensor_id/sensor_name).

oscgonfer and others added 30 commits July 25, 2025 12:31
@oscgonfer
Move PMSA003I to separate class and update AQ telemetry
7f042a0
@oscgonfer
AirQualityTelemetry module not depend on PM sensor presence
835adb2
@oscgonfer
Remove commented line
3b470b7
@oscgonfer
Fixes on PMS class
2f68458
@oscgonfer
Add missing warmup period to wakeUp function
e4903eb
@oscgonfer
Fixes on compilation for different variants
9111f88
@oscgonfer
Add functions to check for I2C bus speed and set it
40af7b8
@oscgonfer @hafu
Add ScreenFonts.h
ff8691d 
Co-authored-by: Hannes Fuchs <hannes.fuchs+git@0xef.de>
@Nashui-Yan @oscgonfer
PMSA003I 1st round test
0dda175
@oscgonfer
Fix I2C scan speed
14eaa3e
@oscgonfer
Fix minor issues and bring back I2C SPEED def
8a811b2
@oscgonfer
Remove PMSA003I library as its no longer needed
ec5a752
@oscgonfer
Merge branch 'master' into enhancement/air-quality-module
f7a9e27
@oscgonfer
Add functional SCD4X
417f5a8
@oscgonfer
Fix screen frame for CO2
00349c3
@oscgonfer
Add admin commands to SCD4X class
8a4f5c1
@oscgonfer
Add further admin commands and fixes.
2609f45
@thebentern
Merge branch 'master' into enhancement/air-quality-module
b23605a
@oscgonfer
Remove unused I2C speed functions and cleanup
a129441 
* Cleanup of SEN5X specific code added from switching branches
* Remove SCAN_I2C_CLOCK_SPEED block as its not needed
* Remove associated functions for setting I2C speed
@oscgonfer
Merge branch 'master' into enhancement/air-quality-module
bb3ec31
@thebentern
Merge branch 'master' into enhancement/air-quality-module
8ec036a
@thebentern
Merge branch 'master' into enhancement/air-quality-module
6d45a86
@thebentern @oscgonfer
Unify build epoch to add flag in platformio-custom.py (meshtastic#7917)
24b3a22 
* Unify build_epoch replacement logic in platformio-custom

* Missed one
@fifieldt @oscgonfer
Fix build error in rak_wismesh_tap_v2 (meshtastic#7905)
0827bd0 
In the logs was:
"No screen resolution defined in build_flags. Please define DISPLAY_SIZE."

set according to similar devices.
@thebentern @oscgonfer
Put guards in place around debug heap operations (meshtastic#7955)
0b900dd 
* Put guards in place around debug heap operations

* Add macros to clean up code

* Add pointer as well
@thebentern @oscgonfer
Cleanup
7b75468
@compumike @oscgonfer
Fix memory leak in NextHopRouter: always free packet copy when removi…
d7ef19b 
…ng from pending
@thebentern @oscgonfer
Formatting
ab09d9b
@thebentern @oscgonfer
Only queue 2 client notification
4aa5e91
@thebentern @oscgonfer
Merge pull request meshtastic#7965 from compumike/compumike/fix-nrf52…
8468bf9 
…-bluetooth-memory-leak

Fix memory leak in `NRF52Bluetooth`: allocate `BluetoothStatus` on stack, not heap
oscgonfer and others added 28 commits January 18, 2026 18:42
@oscgonfer
If one sensor fails, AQ telemetry still reports data
2f95db2
@oscgonfer
Small formatting fix
f6cd9db
@oscgonfer
Add SEN5X to AQI in ScanI2C
b4a2560
@oscgonfer
SCD4X now part of AQ module with template list
52ca090 
* Fixes difference between idle and sleep
* In LowPower, sleep is disabled
* Requires testing for I2C clock comms for commands
@oscgonfer
Remove unnecessary import
c6c97f1
@oscgonfer
Add co2 to serialized AQ metrics
26a05c3
@oscgonfer
Merge branch 'master' into feat/add-scd4x
7b995e1
@oscgonfer
Merge branch 'master' into feat/add-sen5x
46ff9ec
@oscgonfer
Update library dependencies in platformio.ini
958923d
@oscgonfer
Merge branch 'master' into feat/add-sen5x
8d8e116
@oscgonfer
Merge branch 'feat/add-sen5x' into feat/add-scd4x
588a700
@thebentern
Merge branch 'master' into feat/add-scd4x
1e34ee2
@oscgonfer
Merge branch 'master' into feat/add-sen5x
712c43a
@oscgonfer
Fix unitialized variables in SEN5X constructor
d1f1cf4
@oscgonfer
Merge branch 'feat/add-sen5x' into feat/add-scd4x
74aedc9
@oscgonfer
Fix missing import
cdf6183
@oscgonfer
Merge branch 'feat/add-sen5x' into feat/add-scd4x
18cf9f7
@oscgonfer
Fix uninitMemberVars
105206c
@oscgonfer
Fix import error for SCD4X
637e0c6
@oscgonfer
Fix I2CClock logic
66a1fc0 
* Fix not reclocking back to 700000Hz
@oscgonfer
Fix multiple sensors being read simultaneously
5cfbc65 
*  The logic in AQ module is different to the one in EnvironmentTelemetryModule.  In Env module, if any sensor fails to getMetrics, no valid flag for the module. This prevents other sensors to report data.
@thebentern
Fix NimBLE deinit null check
d44ceb6
@thebentern
Merge branch 'develop'
94d7b71
@thebentern
PRs with needs-review still should get bot labeled
03084f6
@renovate
Update libch341-spi-userspace digest to af9bc27 (meshtastic#9472)
b18742c 
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
@thebentern
Merge branch 'master' into feat/add-scd4x
4edfca9
@oscgonfer
Fix pending clock change in PMSA003
7f4f078
@oscgonfer
First proposal for nested telemetry protos in AQ module
95a425b
@oscgonfer oscgonfer marked this pull request as draft January 30, 2026 14:34
@github-actions github-actions bot added the needs-review Needs human review label Jan 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

needs-review Needs human review

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@oscgonfer @Nashui-Yan @thebentern @fifieldt @compumike @WillyJL